Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Final development version of the devloop. #32

Open
wants to merge 2 commits into
base: assert-devloop-improvements
Choose a base branch
from

Conversation

darclander
Copy link

Since my commit directly on the branch created conflicts we decided to make this PR instead. Commit notes below can be seen as an explanation but basically the three final steps of development has been implemented in this branch. The X reruns does not have all the functionality yet but delay should. Timeout "has" the functionality but does not exit the image? properly when timeout has been reached.

Commit notes:
Timeout and delay should now be implemented for the cluster. Have not had enough time to see if it works properly. The logic should be there though. The image is not exited but there is a condition for where it should.

NOTE: Had to revert the previous version of this commit so it became a bit larger than expected. Now the rerun X times is in a dev-mode where after the tests pass a counter goes up, when the counter is larger than 3 it prints a message.

Timeout and delay should now be implemented for the cluster. Have not had enough time to see if it works properly. The logic should be there though. The image is not exited but there is a condition for where it should.

NOTE: Had to revert the previous version of this commit so it became a bit larger than expected. Now the rerun X times is in a dev-mode where after the tests pass a counter goes up, when the counter is larger than 3 it prints a message.
@darclander
Copy link
Author

NOTE: It is important that env.json is present in the specs folder (in the ystack repo, not the one in kubernetes-assert).
It should contain: {"RERUN_TIME": "0", "TIMEOUT": "10000"}.

TIMEOUT: 0 | Timeout will not be used
TIMEOUT: X | Timeout will occur after X/1000 seconds. (It is currently in ms hence the weird format).

RERUN_TIME: X | Will run tests with a rerun time of X seconds.
RERUN_TIME: 0 | Will run tests until they are complete / have passed with a default rerun time of 10 seconds.
RERUN_TIME: != 0 | Will run tests continuously with the rerun time set as the value to the key in the json.

Without a json file it "should" work and the rerun time should be defaulted to 0 and timeout should be default to "infinity".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant